-
-
Notifications
You must be signed in to change notification settings - Fork 396
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Laravel 5.8 Compatibility #731
Conversation
Few things that still need to be done:
|
Just trying this to see what CircleCI does now
This reverts commit 8741b71.
To maintain compatibility with 5.7, you could leave the |
@36864 The interface is not available in L5.7, meaning you'll just get an error saying the Interface does not exist? |
And that's what happens when I try to review code before my morning coffee. Sorry. It's worth pointing out that this will also break compatibility with 5.5, which is the current LTS branch. Maybe we can delay that until the next version, seeing as this package claims to be compatible with latest stable and LTS? |
Delay the ReleaseAt this moment we have not planned a new release of the package yet as there are still errors to take care of. I don't see the fixing of that happening any time soon as we have close to no idea where the exact error is coming from and it's something that still needs to be investigated. This is just a PR that makes the package 5.8 compatible and future-proof (by already removing deprecated things). Breaking CompatibilityThe current code that is in the dev branch is already breaking 5.5 as functionality that got introduced in #711 uses functions that have been added since L5.7. PS: This is just my opinion / view on things |
Fair point about the compatibility, I only brought it up because the readme claims compatibility with latest LTS. I'm just gonna go back to silently appreciating your work now. |
Seems we need to clarify the readme. The new strategy is to stay up to par with the latest release. I'm going to invest some time again this week to assist @ArlonAntonius in getting a release out. |
As per Laravel 5.8 the Defer variable in Service Providers is deprecated. This PR simply fixes the deferrable provider per their requirements, would break compatibility with 5.7 though.